Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📄 요약(Summary)
✍🏼 상세(More)
PR Desciption
아래 사진과 같이 변경된 상태를 저장하기 위한 modifeidStatus 추가
-> 기존 task를 참조하여 status를 가져오는 것은 task의 상태가 변하게 되면 함께 변하기 때문에 잘못된 코드
-> 따라서 시에나 말씀대로 상태의 대한 값을 저장할 수 있는 modifiedStatus 추가
-> modifiedMember도 작업 처리자 변경 시점에 닉네임을 가져와 저장할 수 있지만 닉네임 변경에 대한 부분 고려하여 id를 참조하도록 유지
ChangeTaskController 내의
작업 상태 변경, 작업 처리자 변경, 작업 승인API 호출 시 그에 맞는 작업 히스토리 생성하도록 설정댓글 추가, 댓글 첨부파일 추가 시 작업 히스토리 생성 필요
댓글 삭제가 가능하다면 댓글 삭제 시점에 히스토리를 삭제하거나 히스토리 조회시 삭제된 댓글은 히스토리로 조회하지 않는 로직 설정 필요
Requirements for Reviewer
PR Log
새롭게 배운 것
고민 중인 사항
첨부 자료
Requirements for Reviewer
✅ 체크리스트(Checklist)
🚪 이슈 번호(Issue numbers)
Closes #168